Skip to content

[SPARK-10190] Fix NPE in CatalystTypeConverters Decimal toScala converter#8401

Closed
JoshRosen wants to merge 1 commit into
apache:masterfrom
JoshRosen:SPARK-10190
Closed

[SPARK-10190] Fix NPE in CatalystTypeConverters Decimal toScala converter#8401
JoshRosen wants to merge 1 commit into
apache:masterfrom
JoshRosen:SPARK-10190

Conversation

@JoshRosen

Copy link
Copy Markdown
Contributor

This adds a missing null check to the Decimal toScala converter in CatalystTypeConverters, fixing an NPE.

@rxin

rxin commented Aug 24, 2015

Copy link
Copy Markdown
Contributor

LGTM

@SparkQA

SparkQA commented Aug 24, 2015

Copy link
Copy Markdown

Test build #41473 has finished for PR 8401 at commit 91f3217.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public static class AppExecId
    • public static class StoreVersion

@rxin

rxin commented Aug 24, 2015

Copy link
Copy Markdown
Contributor

I've merged this. Thanks!

@asfgit asfgit closed this in d7b4c09 Aug 24, 2015
asfgit pushed a commit that referenced this pull request Aug 24, 2015
…rter

This adds a missing null check to the Decimal `toScala` converter in `CatalystTypeConverters`, fixing an NPE.

Author: Josh Rosen <joshrosen@databricks.com>

Closes #8401 from JoshRosen/SPARK-10190.

(cherry picked from commit d7b4c09)
Signed-off-by: Reynold Xin <rxin@databricks.com>
@JoshRosen JoshRosen deleted the SPARK-10190 branch August 24, 2015 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants